Skip to content

Conversation

@innocenzi
Copy link
Member

No description provided.

@innocenzi innocenzi force-pushed the docs/processes branch 2 times, most recently from 54ceb4f to 925c79f Compare August 15, 2025 01:07
@innocenzi innocenzi marked this pull request as draft August 15, 2025 01:12
@innocenzi
Copy link
Member Author

innocenzi commented Aug 15, 2025

I have no clue why the isolated tests fail, tried to debug the CI but can't get it to display more information... will try again later

@innocenzi
Copy link
Member Author

That was the weirdest bug I've ever seen. When trying to use the same environment as the CI, I had to install deps in the packages/process directory.

Doing so made a new error appear on the ProcessPoolResults class: I had to implement serialize and unserialize, I still don't know why. The class implemented our ArrayInterface interface, which in turn implements Iterator, ArrayAccess and Countable.

As far as I know, none of them implement Serializable, so I'm clueless. I implemented these three interfaces directly instead of going through ArrayInterface and the error disappeared? Even though it's namespaced, I'm assuming there is a resolution issue with the interface name "ArrayInterface".

@innocenzi innocenzi marked this pull request as ready for review August 15, 2025 17:46
@innocenzi innocenzi merged commit 62d2c1a into 2.x Aug 15, 2025
137 of 138 checks passed
@innocenzi innocenzi deleted the docs/processes branch August 15, 2025 17:46
Bapawe pushed a commit to Bapawe/tempest-framework that referenced this pull request Aug 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants